Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable NCAAF Team to be directly accessible #405

Merged
merged 1 commit into from
Apr 26, 2020

Conversation

roclark
Copy link
Owner

@roclark roclark commented Apr 25, 2020

Instead of requiring users to go through the Teams class to get a specific team, the NCAAF modules now enable a specific team to be directly queried by using the Team class. This reduces computational complexity by removing the need to instantiate every team while also
making it more intuitive for users.

Related to #360

Signed-Off-By: Robert Clark robdclark@outlook.com

@roclark roclark added the enhancement New feature or request label Apr 25, 2020
@roclark roclark added this to the Release 0.6.0 milestone Apr 25, 2020
@roclark roclark self-assigned this Apr 25, 2020
@roclark roclark force-pushed the direct-ncaaf-team-module branch 4 times, most recently from f2831b5 to b4c9999 Compare April 26, 2020 16:21
Instead of requiring users to go through the Teams class to get a
specific team, the NCAAF modules now enable a specific team to be
directly queried by using the Team class. This reduces computational
complexity by removing the need to instantiate every team while also
making it more intuitive for users.

Signed-Off-By: Robert Clark <robdclark@outlook.com>
@codecov
Copy link

codecov bot commented Apr 26, 2020

Codecov Report

Merging #405 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #405   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files          51       52    +1     
  Lines       10665    10683   +18     
=======================================
+ Hits        10662    10680   +18     
  Misses          3        3           
Impacted Files Coverage Δ
sportsreference/ncaaf/ncaaf_utils.py 100.00% <100.00%> (ø)
sportsreference/ncaaf/teams.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b9c132...8f6eb9a. Read the comment docs.

@roclark roclark merged commit 0ba4b6f into master Apr 26, 2020
@roclark roclark deleted the direct-ncaaf-team-module branch April 26, 2020 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant